Skip to content

Revert Count Schedules API#3203

Merged
rossedfort merged 4 commits intomainfrom
revert-schedule-count
Feb 27, 2026
Merged

Revert Count Schedules API#3203
rossedfort merged 4 commits intomainfrom
revert-schedule-count

Conversation

@laurakwhit
Copy link
Collaborator

Description & motivation 💭

Reverts use of new CountSchedules API introduced in #3100.

Screenshots (if applicable) 📸

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

@vercel
Copy link

vercel bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
holocene Ready Ready Preview, Comment Feb 27, 2026 10:10pm

Request Review

@laurakwhit laurakwhit force-pushed the revert-schedule-count branch from 02b8e84 to b4b6388 Compare February 27, 2026 04:53
@temporalio temporalio deleted a comment from temporal-cicd bot Feb 27, 2026
@temporal-cicd
Copy link
Contributor

temporal-cicd bot commented Feb 27, 2026

Warnings
⚠️

📊 Strict Mode: 1 error in 1 file (0.1% of 1110 total)

src/lib/services/workflow-counts.ts (1)
  • L34:3: Function lacks ending return statement and return type does not include 'undefined'.

Generated by 🚫 dangerJS against 291eeca

@Alex-Tideman
Copy link
Collaborator

I'd want to verify that the old api still works on the latest server version. It seemed like it was being deprecated but I don't know what version that would be

@laurakwhit
Copy link
Collaborator Author

Closing in favor of #3205

@laurakwhit laurakwhit closed this Feb 27, 2026
@rossedfort rossedfort reopened this Feb 27, 2026
@rossedfort rossedfort merged commit ee254fb into main Feb 27, 2026
16 checks passed
@rossedfort rossedfort deleted the revert-schedule-count branch February 27, 2026 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants